-
Notifications
You must be signed in to change notification settings - Fork 55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes to policy storage and policy attachment files #604
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me, thank you Danny!
Co-authored-by: Robin Beck <stellarsquall@users.noreply.github.com>
Co-authored-by: Robin Beck <stellarsquall@users.noreply.github.com>
Co-authored-by: Robin Beck <stellarsquall@users.noreply.github.com>
@dannyjknights do we still want to merge this? Might need a new push to rerun the tests but otherwise the added examples seem good |
Yes please :) I thought it was already done to be honest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah I see the gen_delta is failing, the reason for this is that we cannot update the docs directly since they are generated instead we should create an corresponding file in https://github.com/hashicorp/terraform-provider-boundary/tree/main/examples/resources and then run make docs
scope_policy_attachment.md
had no description or example usage for attaching a storage policy. Thepolicy_storage.md
had no example usage for creating a storage policy.